Skip to content

Add new section 'Supporting a new SoC' #99

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mbrossard
Copy link

No description provided.

Copy link

@rursprung rursprung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is a random passing-by review, i don't have a stake in this but saw it and was positively surprised by it!

one thing i'm wondering: it seems that a lot of HAL activity is now centered around embassy (with the exception of e.g. esp-hal). i'm wondering if this should be explicitly pointed out so that vendors could opt to implement their PACs & HALs directly in embassy rather than having them separate? OTOH embassy is not a WG project, thus this might be the wrong place for this?

recommend reaching out to the Embedded Rust Working Group (REWG) leads. They
can provide valuable insights and support to help you navigate the process
effectively.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think step 0 is missing: support for the target architecture in LLVM (and step 0b: rustc using an LLVM version which contains said support).
the alternative is the manufacturer maintaining an LLVM & rustc fork (until things get upstreamed, anyway), see what espressif is doing with espup for the xtensa arch.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This would fit in a new section I am still working, which a suggested overall flow. This was an idea from @diondokter and @jamesmunns mentioned the How do I add support for a new microcontroller to embassy? he wrote in Embassy documentation.

@9names
Copy link

9names commented Jun 17, 2025

vendors could opt to implement their PACs & HALs directly in embassy rather than having them separate?

This is not required by embassy, and vendors typically prefer to keep their codebases in their own org/repo so they can control access to them, there's no doubts about ownership, etc.

Also: maybe ask dirbaio before you start suggesting that everyone shove their code in his project's repo?

@rursprung
Copy link

Also: maybe ask dirbaio before you start suggesting that everyone shove their code in his project's repo?

sorry, this wasn't meant as a "hey, everyone should be doing this!" and more as a "is it the idea that this could/should be done?". sorry if that didn't come across like that

CC @Dirbaio

@felipebalbi
Copy link

This looks great to me, thanks for updating.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants